Skip to content

stash/GuessRecipeVersion: Fix recipes without version#363

Merged
priv-kweihmann merged 1 commit into
priv-kweihmann:mainfrom
michalsieron:stash-guessrecipeversion
Feb 24, 2026
Merged

stash/GuessRecipeVersion: Fix recipes without version#363
priv-kweihmann merged 1 commit into
priv-kweihmann:mainfrom
michalsieron:stash-guessrecipeversion

Conversation

@michalsieron
Copy link
Copy Markdown
Contributor

When a recipe didn't have version in its name, this function was returning its entire name instead of either an empty value or Yocto default "1.0".

Comment thread oelint_parser/cls_stash.py Outdated
@michalsieron michalsieron force-pushed the stash-guessrecipeversion branch from 0b286e0 to f747b90 Compare February 23, 2026 14:59
When a recipe didn't have version in its name, this function was
returning its entire name instead of either an empty value or Yocto
default "1.0".

I changed it to return None if no version can be extracted from the file
name. I also added a new test in tests/test_expand.py to check multiple
scenarios of versioned/unversioned recipe files with or without PV
defined in them.

Signed-off-by: Michal Sieron <michalwsieron@gmail.com>
@michalsieron michalsieron force-pushed the stash-guessrecipeversion branch from f747b90 to e5134d6 Compare February 23, 2026 15:00
@michalsieron
Copy link
Copy Markdown
Contributor Author

@priv-kweihmann I think I addressed your points in cls_stash.py. Added new test as requested with several test cases inside.

@priv-kweihmann priv-kweihmann merged commit aef11c9 into priv-kweihmann:main Feb 24, 2026
5 checks passed
@michalsieron michalsieron deleted the stash-guessrecipeversion branch February 27, 2026 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants